Fix portal path handling

This was broken when I recently introduced this helper
function.

rendernodepaintable: Fix clipping

We need to translate before clipping, otherwise we clip in totally the
wrong place.

iconhelper: Fix drawing of empty and paintable helpers

Empty helpers did try to draw a NULL paintable (not good) and in the
non-null case code used the wrong width/height.

Merge branch 'gtkbbox-doc-fix' into 'master'

Minor fixes in documentation of gtkbbox

See merge request GNOME/gtk!318

Minor fixes in documentation of gtkbbox

Updated Danish translation of gtk-properties

Updated Danish translation of gtk

Updated Danish translation of gtk

Updated Danish translation of gtk-properties

Adwaita: Regenerate CSS to get window.devel styles

This step was missed before, again.

SASS 3.6 emits rgba(0, 0, 0, 0) instead of transparent, so it wants to
change those too, but that patch was only committed in March and isn't
being backported to the previous stable, so I don't know if others'
versions will do the same - so until it's shown that anyone else (A) is
regenerating CSS and (B) also has 3.6, I'm skipping those changes. See:
https://github.com/sass/libsass/commit/c287f312ac9735aa274bbce56762391ca348cad9

Update Polish translation

GtkApplication: Improve the docs

Expand the docs for ::register-session a bit.

Avoid a pointless goto

We can just return here.

Add a helpful comment

GtkApplication: track screensaver state

A number of applications want to track the state of the screensaver.
Make this information available as a boolean property. We only listen
for state changes when ::register-session is set to TRUE.

This is implemented for unsandboxed D-Bus access by talking
directly to org.gnome.ScreenSaver or org.freedesktop.ScreenSaver,
and for sandboxed D-Bus by using a (new) portal API.
A Quartz implementation is missing.

GtkApplication: Respect GTK_USE_PORTAL

When the environment variable is set, don't connect
to the session manager, but instead rely on the
inhibit portal.

file chooser portal: use request path utilities

Less code duplication, more sticky toffee!

print portal: use request path utilities

Less code duplication, more cookies!

color picker portal: Add request path utility

Less code duplication, more cake!

Add utility functions for portal paths

The paths that we create for requests and sessions
need some icky code to create. Keep it in one place.

Updated Czech translation

Updated Czech translation

Merge branch 'selection-mode-separator' into 'master'

Adwaita: Add color to separator.selection-mode

See merge request GNOME/gtk!309

Add a missing cast

This fixes a warning introduced in the previous commit.

Updated Lithuanian translation

The file chooser is no box anymore

Don't treat it as one, it does not like it.

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1297

Update Swedish translation

FileChooserWidget: Don't show recent items with private hint set

Update Polish translation

Fix a typo

Its Hieroglyphs!

Closes: #1292

Merge branch 'revealer-clipping' into 'master'

revealer: Only clip child when animating

See merge request GNOME/gtk!301

Update Brazilian Portuguese translation

Revert "paned: Use a GtkIcon as separator"

This reverts commit 749ef4d71ca018a18fd608bf3b2e4c022727e2ae.

The GtkIcon and GtkGizmo measure code is different, the former uses
-gtk-icon-size.

box: Remove gtk_box_get_size

One function to measure the box in the opposite of its internal
direction is enough.

box: Rename avail_size to for_size everywhere

So we call it like we call it everywhere else.

box: Simplify compute_size_for_orientation

testoverlaystyleclass: Fix CSS snippet

remove tests/testinput

It's broken and almost the exact same thing as the drawing demo in
gtk4-demo.

Adwaita: Add color to separator.selection-mode

This makes separators to look good when separating two header bars in
selection mode.

https://gitlab.gnome.org/GNOME/gtk/issues/1286

Merge branch 'robertroth/readme-fix' into 'master'

docs: README.commits converted to markdown

See merge request GNOME/gtk!304

docs: README.commits converted to markdown

Update German translation

fishbowl: Use a GtkPicture for videos

menuitem: Remove duplicate include

picture: Fix doc typo

spinner: Remove private struct

Since priv->active is equivalent to the widget state being CHECKED, we
can as well use that everywhere.

colorswatch: Remove dead assignment

image: Fix gtkdoc property syntax

paned: Use a GtkIcon as separator

And save a few lines that way.

menubutton: Remove outdated line

.menu-button is not a style class we use anywhere.

renderborder: Remove unused function

renderborder: Pull some locals into the closest scope

Especially the bounds graphene_rect_t, which is unused in the
non-border-image case.

checkbutton: Remove snapshot implementation

It just does when the default GtkWidget implementation does anyway:
snapshot all child widgets

radiobutton: Save signal IDs like everything else

fixed: Remove unused struct member

revealer: Only clip child when animating

Currently, GtkRevealer clips the child if the transition type is
sliding, regardless of whether the animation had already ended. An
example where that is a problem would be in Nautilus: the file
operations popover button is animated on reveal to draw attention, but,
given that the button is in turn stashed inside a revealer with a
sliding animation, things suddenly fall apart.

entry completion: Avoid critical warnings

We must not call move-to-rect unless we have
a transient parent.

Merge branch 'fix-shifted-scrolling' into 'master'

gtkscrolledwindow: Consider shift key presses when decelerating

Closes #770

See merge request GNOME/gtk!286

gesturestylus: Add missing array g-i annotation for *_get_axes params

Update POTFILES.in

Merge branch 'kill-subsurface' into 'master'

Kill subsurfaces

See merge request GNOME/gtk!299

Move tools to a subdirectory

A small step towards splitting up gtk/

gdk: Drop GDK_SURFACE_SUBSURFACE

None of the backends supports this type anymore.

wayland: drop subsurfaces

We don't use them in GTK+ anymore.

Drop gtk_window_set_use_subsurface

This is no longer used.

entry completion: stop using subsurfaces

Instead, use a popup and gdk_surface_move_to_rect.
I have not tried to reproduce all details of the old
positioning logic, but moving the popup above/below
the entry works as before.

tooltip: Implement positioning using gdk_window_move_to_rect()

In order to make tooltip positioning portable, make use of the
move_to_rect API. Some semantical changes are made, as identical
semantics cannot be implemented using the move-to-rect API.

Primarily the implemented semantics are:

Position the tooltip in the center pixels slightly below (defaults to 4
units below) the tooltipped widget. This is always the case for keyboard
driven tooltips; the case where it tries to avoid the pointer cursor is
not implemented.

For pointer position triggered tooltips, implement the following
additional semantics:

Use the current cursor size to determine the padding used to enlarge the
anchor rectangle. This is to try to avoid the cursor overlapping the
tooltip.

If the anchor rectangle is too tall (meaning if we'd be constrained
and flip on the Y axis, it'd flip too far away from the originally
intended position), rely only on the pointer position to position the
tooltip. The approximate pointer cursor rectangle is used as a anchor
rectangle. Ideally we should use the actual pointer cursor rectangle
(image used as well as hotspot coordinate), but we don't have API to
get that information.

If the anchor rectangle isn't to tall, just make sure the tooltip isn't
too far away from the pointer position on the X axis.

Closes: #134
Closes: #432
Closes: #574
Closes: #579
Closes: #878

gdk/wayland: Avoid idempotent wl_subsurface.set_position calls

These may not result on wl_surface.frame callbacks, yet we do trigger
a frame clock tick that would get stuck on the lack of such callback.

https://bugzilla.gnome.org/show_bug.cgi?id=784314
https://gitlab.gnome.org/GNOME/gtk/issues/844

Closes: #844

gdk/wayland: Always map windows 'moved-to-rect' as popups

Only popups can make use of the move-to-rect semantics so it makes no
sense to try anything other surface type.

gdk/wayland: Don't remap non-subsurface as subsurface

Let's just use the fact that a window was mapped as a subsurface to
remap it above another transient parent instead of relying on the more
complicated 'should-map-as-subsurface' helper function.

Merge branch 'win32-smooth-scrolling-gtk4' into 'master'

GDK W32: Support smooth scrolling (GTK4)

See merge request GNOME/gtk!298

menu: Remove key_controller member

Unused.

menu: Remove initially_pushed_in flag

Unused.

menu: Remove seen_item_enter flag

Unused.

menu: Remove have_position flag

It's always FALSE.

menu: Remove an unnecessary gtk_widget_destroy() call

The unref will already properly free the menu item's resources.

GDK W32: Support smooth scrolling

Set delta_x or delta_y for GdkScrollEvent.
HIWORD (wParam) in WM_MOUSE(H)WHEEL is the scroll delta.
A delta value of WHEEL_DELTA (which is 120) means scrolling
one full unit of something (for example, a line).

The delta should also be multiplied by the value that the
SystemParametersInfo (SPI_GETWHEELSCROLL(LINES|CHARS), 0, &value, 0)
call gives back, unless it gives back 0xffffffff, in which case
it indicates that scrolling is page- or screen-based, not line-based
(GDK doesn't support that at the moment).

Also, all deltas should be inverted, since MS sends negative deltas
when scrolling down (rotating the wheel back, in the direction of
the user).

With deltas set the mode should be set to GDK_SCROLL_SMOOTH.

Fixes issue 1263.

Don't fail the build if c++ isn't found

The intention of this check was to skip the keyword
test if no c++ compiler is found. But the meson
docs say that add_languages() will abort unless we
pass required: false.

Merge branch 'fix-shell-colorpicker' into 'master'

colorpickershell: Unpack the tuple returned from PickColor()

See merge request GNOME/gtk!296

print portal: don't leak window handles

Pointed out by Christian Persch.

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1274

print portal: Plug a memory leak

Pointed out by Christian Persch.

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1271

Use defines for bus names

This helps eradicate typos (we had some here).

Closes https://gitlab.gnome.org/GNOME/gtk/issues/1267

Merge branch 'sort' into 'master'

updateiconcache: Sort list of entries

See merge request GNOME/gtk!297

font chooser: Remove an unneeded ifdef

We can just always create the axes hashtable.

updateiconcache: Sort list of entries

because filesystem readdir order is indeterministic.

Without this patch, building openSUSE's balsa package
had variations between builds in /usr/share/balsa/icon-theme.cache

Merge branch 'master' into 'master'

window: Fix memory leak

See merge request GNOME/gtk!295

colorpickershell: Unpack the tuple returned from PickColor()

When calling PickColor on org.gnome.Shell, we get back an "a{sv}", which
GDBus provides to us as "(a{sv})".

At the minute we're not unpacking this tuple, and so picking fails with
messages like:

  GLib-CRITICAL **: 13:38:19.439: g_variant_lookup_value: assertion 'g_variant_is_of_type (dictionary, G_VARIANT_TYPE ("a{s*}")) || g_variant_is_of_type (dictionary, G_VARIANT_TYPE ("a{o*}"))' failed

  Gtk-WARNING **: 13:38:19.439: Picking color failed: No color received

Let's unpack it.

window: Fix memory leak

https://gitlab.gnome.org/GNOME/gtk/issues/1268

Update Turkish translation

Update Polish translation

Merge branch 'gtk-4-improve-error-message' into 'master'

gtkplacessidebar: Improve error message when unlocking volume fails

See merge request GNOME/gtk!267

Update Polish translation

migration guide: Mention GtkEventBox

Its gone.

Closes: https://gitlab.gnome.org/GNOME/gtk/issues/1243

README: Fix Gdk doc website link

README: Document nightly docs location

Useful for exposing the documentation early.

gtkplacessidebar: Improve error message when unlocking volume fails

Updated Lithuanian translation

widget: Rename width and height member to {width,height}_request

Because that's much closer to the meaning of those values.

main: Pass toplevel_widget directly to GtkWidget API

We already have the toplevel as GtkWidget: toplevel_widget.

sizerequestcache: Inline out-var assignments

The additional assignment to the old result variable just adds an
indirection even though we know the point where we assign it in all
cases. Just pass the values out and return in those cases instead.